Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

On OSX revert back to terminal hack to avoid a readline bug #15

Merged
merged 1 commit into from Sep 19, 2012

Conversation

davglass
Copy link
Contributor

Ran into an issue with one of my apps, I believe it's a readline/process issue, but this reverts the ctrl+u clearing back to \r only for OSX.

This still works on Windows and everywhere else, but this fixes an issue when you have several child processes sharing the same stdin, the ctrl+u actually looks to be getting called once per child that's bound and not just once. So the progress bar will write one line and delete 10 (one per child) before it..

Totally screwy and will take a ton of debugging, but this works for now.

@tj
Copy link
Member

tj commented Sep 19, 2012

haha, node is so portable!!, sucks that windows terminals dont handle \r correctly

tj added a commit that referenced this pull request Sep 19, 2012
On OSX revert back to terminal hack to avoid a readline bug
@tj tj merged commit f0e3963 into visionmedia:master Sep 19, 2012
@davglass
Copy link
Contributor Author

I know all too well ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants